Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Snapcast: Add configuration options #1692

Merged
merged 12 commits into from
Oct 5, 2024
Merged

Conversation

SantiagoSotoC
Copy link
Contributor

We have a new contributor, @whc2001 thank you for your help 🎉🎉

Some configuration options and some related details are fixed.

Fixes the problem with tts cutting off in the middle of the time, but there is still a problem with very short tts, like “Hi” where it just stops the music that is playing and comes back with the music after a few seconds and you never hear the chime or the “Hi”.

@SantiagoSotoC
Copy link
Contributor Author

@whc2001 are you using pre commit ?

@whc2001
Copy link

whc2001 commented Oct 2, 2024

@whc2001 are you using pre commit ?

Ah sorry I think it's not properly configured on the Windows machine I used to push

@marcelveldt
Copy link
Member

Ah sorry I think it's not properly configured on the Windows machine I used to push

Windows is not recommended for python development, especially not for MA development.
Best is to use MacOS or Linux. Dev container could potentially work but has caveats.

Copy link
Member

@marcelveldt marcelveldt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, nice work!

@marcelveldt
Copy link
Member

Want to merge this now or do you want to adjust those category labels first ?

@SantiagoSotoC
Copy link
Contributor Author

Want to merge this now or do you want to adjust those category labels first ?

The categories are already set but we are going to do the localization here, otherwise everything will remain without description

@SantiagoSotoC
Copy link
Contributor Author

@whc2001 I am already editing the frontend en.json

@SantiagoSotoC
Copy link
Contributor Author

@whc2001 I am already editing the frontend en.json

music-assistant/frontend#673

@whc2001
Copy link

whc2001 commented Oct 3, 2024

@whc2001 I am already editing the frontend en.json

music-assistant/frontend#673

Just got there, I'll check

EDIT: Seems like my understanding of the lang file is a little off, what I did is something like

"snapcast_server_built_in_initial_volume": {
    "label": "Initial volume",
    "description": "Volume assigned to new snapclients [percent]"
},

But seems like your method is working well so I think we are good.

@whc2001
Copy link

whc2001 commented Oct 3, 2024

Already got my latest change in music-assistant/frontend#674 and #1695

@marcelveldt marcelveldt merged commit 4021217 into main Oct 5, 2024
7 checks passed
@marcelveldt marcelveldt deleted the snapcast_add_idle_threshold branch October 5, 2024 08:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants